Skip to content

Conversation

laileni-aws
Copy link
Contributor

Problem

  • Added new UX to inform user about migration of /agents.

Solution

Old JB plugin behavior with new flare version

JB.Old.plugin.with.new.flare.mov

New JB plugin behavior with new flare version

JB.new.plugin.with.new.flare.mov

Checklist

  • My code follows the code style of this project
  • I have added tests to cover my changes
  • A short description of the change has been added to the CHANGELOG if the change is customer-facing in the IDE.
  • I have added metrics for my changes (if required)

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

@laileni-aws laileni-aws requested a review from a team as a code owner September 15, 2025 23:22
Copy link
Contributor

@manodnyab manodnyab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we add a changelog?

@@ -1,14 +1,13 @@
// Copyright 2024 Amazon.com, Inc. or its affiliates. All Rights Reserved.
// SPDX-License-Identifier: Apache-2.0
/*!
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need these connector files?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No I wrote a TODO to remove all these in next PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't want to this PR to be large and this defeats the purpose instead I will raise a separate PR cleaning all the / agents files

@laileni-aws
Copy link
Contributor Author

can we add a changelog?

I don't think we need changeLog as we already added one for Migration of / agents before. This PR is to remove the UX which was added as part of this transition.

@laileni-aws laileni-aws enabled auto-merge (squash) September 16, 2025 17:11
auto-merge was automatically disabled September 16, 2025 17:43

Pull request was closed

@laileni-aws laileni-aws reopened this Sep 16, 2025
@laileni-aws laileni-aws merged commit 785a39e into aws:main Sep 16, 2025
15 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants